Bump dependencies and tighten pr-open command#13
Conversation
There was a problem hiding this comment.
Pull request overview
Routine dependency refresh across the monorepo (TypeScript 6, Prettier/Vitest/tooling bumps, app deps like hono/openai/wrangler), plus an update to the /pr-open Claude command to avoid pushing on behalf of the user and to add a “branch exists on remote” pre-flight check.
Changes:
- Bump core tooling versions (TypeScript 6.0.2, Prettier 3.8.2, Vitest 4.1.4, typescript-eslint 8.58.1,
@types/node25.6.0,@eslint/compat2.0.5). - Bump app/runtime dependencies for
apps/operator(hono 4.12.12, openai 6.34.0, wrangler 4.81.1, workers-types 4.20260412.1). - Tighten
.claude/commands/pr-open.mdto nevergit pushand to require the branch to already exist on the remote.
Reviewed changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tooling/prettier/package.json | Bumps Prettier/TypeScript/@types/node for the shared Prettier tooling package. |
| tooling/eslint/package.json | Bumps typescript-eslint/@eslint/compat and aligns toolchain versions (TS/Prettier/@types/node). |
| package.json | Bumps root Prettier to match the refreshed toolchain. |
| apps/operator/package.json | Bumps operator app deps/devDeps (hono/openai/wrangler/workers-types/TS toolchain). |
| packages/logger/package.json | Aligns dev toolchain versions (Prettier/TypeScript/Vitest). |
| packages/http-client/package.json | Aligns dev toolchain versions (Prettier/TypeScript/Vitest). |
| pnpm-lock.yaml | Updates lockfile to reflect all dependency bumps and transitive changes. |
| .claude/commands/pr-open.md | Updates command flow to avoid pushing and adds a remote-branch pre-flight requirement. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ae9896ae44
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
What
eslint/@eslint/jsstay on 9.39.1 —eslint-plugin-importdoesn't yet declare eslint-10 peer support./pr-openslash command no longer pushes for the user (push step removed, branch-on-remote pre-flight added, copy reworded to third-person).How to test
pnpm install
pnpm typecheck
pnpm lint
pnpm test
pnpm format:check
Security review
Dependency-only bumps; no auth, network, or data-handling code changed.